-
Notifications
You must be signed in to change notification settings - Fork 34
PHP Comprehensive Coverage #282
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
* DOCSP-47058: Shift TOC * edits * issues and help, edits * fix errors * redirects * landing * RR feedback * RR feedback 2
* DOCSP-47057: Network compression * delete * edits * MM feedback
* DOCSP-47055: Collation * add to each page * edit * edits * list format * SA feedback
* DOCSP-47060: Causal consistency * edits
* Check code formatting * fix highlighting
* DOCSP-47059: Work with BSON * edits * RR feedback * enum ex * feedback
* DOCSP-50222: Reorganize auth pages * edits * fix * snooty
* DOCSP-47065: Load balancing * edit * word * RR feedback
* DOCSP-47056: Monitoring * fixes * fix * wording * table * widths * SA feedback * PV feedback
* DOCSP-47071: Extended JSON * edits * vale * feedback
* DOCSP-47072: Client parameters * edits * fix
* DOCSP-47063: Logging * edits * word * feedback * code edit * fix * JT feedback * edit * spacing * gh link * JT feedback 2 * RR feedback
* DOCSP-50185: Connection troubleshooting * edits * remove sharedinclude * fix * wording * fix * MM feedback
* DOCSP-50220: Break up FAQ * edits * RM feedback * AB feedback * fix highlight
* DOCSP-50960: Install with pie * edits * RR feedback * JT feedback
* DOCSP-51123: CC edits and cleanup * build * composer * code errors * commas * more phpcs errors * edits * phpcs * code fixes * more phpcs * fix * RR feedback
* DOCSP-50772: isView() * fix manual link Co-authored-by: Nora Reidy <[email protected]> --------- Co-authored-by: Nora Reidy <[email protected]>
✅ Deploy Preview for docs-php-library ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
🔄 Deploy Preview for docs-php-library processing
|
$info = new CollectionInfo([ | ||
'name' => 'foo', | ||
'type' => 'view', | ||
'info' => ['readOnly' => true], | ||
]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm aware this may be consistent with other examples for various CollectionInfo
methods, but this isn't real-world usage of the class. Typically, we'd expect users to obtain CollectionInfo
instances by calling $database->listCollections()
. If we want to show a functional example, we should consider creating a view, then specifically printing the CollectionInfo
instance for that view.
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-51123
See this slack message for reviewing context.
Staging Links
Self-Review Checklist